Skip to content

Conversation

@michael-s-molina
Copy link
Member

SUMMARY

Moves Card to the components folder.

This work is part of SIP-61

@rusackas @junlincc

TEST PLAN

1 - Execute all tests
2 - All tests should pass

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@codecov
Copy link

codecov bot commented Apr 14, 2021

Codecov Report

Merging #14139 (aa23536) into master (c760030) will increase coverage by 2.59%.
The diff coverage is 40.00%.

❗ Current head aa23536 differs from pull request most recent head 4054ccf. Consider uploading reports for the commit 4054ccf to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14139      +/-   ##
==========================================
+ Coverage   77.01%   79.61%   +2.59%     
==========================================
  Files         954      946       -8     
  Lines       48071    47966     -105     
  Branches     5973     6083     +110     
==========================================
+ Hits        37024    38188    +1164     
+ Misses      10850     9657    -1193     
+ Partials      197      121      -76     
Flag Coverage Δ
cypress 56.38% <25.00%> (?)
presto ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rset-frontend/src/SqlLab/components/QueryTable.jsx 66.66% <ø> (ø)
...et-frontend/src/SqlLab/components/TableElement.jsx 89.02% <ø> (ø)
...erset-frontend/src/datasource/DatasourceEditor.jsx 66.38% <ø> (ø)
superset-frontend/src/components/Card/index.tsx 25.00% <25.00%> (ø)
superset-frontend/src/common/components/index.tsx 100.00% <100.00%> (ø)
superset-frontend/src/filters/components/index.ts 0.00% <0.00%> (-100.00%) ⬇️
...tend/src/filters/components/Time/transformProps.ts 0.00% <0.00%> (-100.00%) ⬇️
...d/src/filters/components/Time/TimeFilterPlugin.tsx 0.00% <0.00%> (-100.00%) ⬇️
...ntend/src/filters/components/Range/controlPanel.ts 25.00% <0.00%> (-75.00%) ⬇️
...rset-frontend/src/filters/components/Time/index.ts 33.33% <0.00%> (-66.67%) ⬇️
... and 315 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c760030...4054ccf. Read the comment docs.

@junlincc junlincc added the sip:61:folder-org Related to the organization of the folders as of SIP 61 label Apr 15, 2021
Copy link
Member

@pkdotson pkdotson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@rusackas rusackas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trying to avoid using SupersetTheme directly, but the rest of it LGTM.

@michael-s-molina michael-s-molina force-pushed the card-to-components-folder branch from caf500a to 9578d1f Compare April 19, 2021 14:27
@michael-s-molina michael-s-molina force-pushed the card-to-components-folder branch from 9578d1f to 4054ccf Compare April 22, 2021 13:36
Copy link
Member

@rusackas rusackas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rusackas rusackas merged commit a52ff13 into apache:master Apr 29, 2021
amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request May 3, 2021
* master: (38 commits)
  refactor(native-filters): allow cascading only for filter_select (apache#14441)
  test(maximize-chart): Add tests to maximize chart action (apache#14371)
  fix: fixing mysql error message (apache#14416)
  feat: Logic added to limiting factor column in Query model (apache#13521)
  change relationship (apache#14435)
  fix: bootstrap data permissions (apache#14348)
  fix: parse simple string error message values (apache#14360)
  chore: add stack trace to all calls of logger.error (apache#14382)
  update README with new docs and recordings (apache#14432)
  Renamed impyla from implya in impala.mdx and Renamed PIP package impyla from impala in index.mdx (apache#14425)
  fix(native-filters): fix filter scope error (apache#14426)
  feat: Adding limiting_factor column to Query model (apache#14234)
  feat: Add etag caching to dashboard APIs (apache#14357)
  chore: Moves Card to the components folder (apache#14139)
  feat: Dynamic imports for the Icons component (apache#14318)
  feat: Support env vars configuration for WebSocket server (apache#14398)
  fix: SQLLab role permissions (apache#14372)
  fix(native-filters): always show filters without dataset (apache#14409)
  fix error getting partitionQuery from table.partition (apache#14369)
  refactor: Boostrap to AntD - Tabs (apache#14048)
  ...
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 First shipped in 1.2.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels sip:61:folder-org Related to the organization of the folders as of SIP 61 size/M 🚢 1.2.0 First shipped in 1.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants